web/sdk/admin/views/organizations/details/layout/navbar.tsx (2)

241-243: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value

Fallback to organizationId if the parsed segment is empty.

While currentPath usually includes the organization slug, adding an explicit fallback protects against edge cases where the extracted segment evaluates to an empty string (e.g., if the path exactly matches orgPrefix without a trailing segment).

♻️ Proposed refactor

   const orgSegment = currentPath.startsWith(orgPrefix)
-    ? currentPath.slice(orgPrefix.length).split("/")[0]
+    ? currentPath.slice(orgPrefix.length).split("/")[0] || organizationId
     : organizationId;

---

319-337: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value

Extract navigation paths to reduce duplication.

You can extract the string literals for the paths into variables to avoid duplicating them between the href and onNavigate callbacks.

♻️ Proposed refactor

@@ -311,27 +311,28 @@
     search.onChange(value);
   }
 
+  const listPath = `/${adminPaths.organizations}`;
+  const detailsPath = `${listPath}/${organization?.name || organization?.id}`;
+
   return (
     <nav className={styles.navbar}>
       <Flex gap={4} align="center">
         <Breadcrumb size="small">
           <Breadcrumb.Item
-            href={`/${adminPaths.organizations}`}
+            href={listPath}
             onClick={(e) => {
               e.preventDefault();
-              onNavigate(`/${adminPaths.organizations}`);
+              onNavigate(listPath);
             }}
             leadingIcon={<OrganizationIcon />}
           >
             {t.organization({ plural: true, case: "capital" })}
           </Breadcrumb.Item>
           <Breadcrumb.Separator />
           {/* Navigate client-side so we don't full-reload and lose the held org id. */}
           <Breadcrumb.Item
-            href={`/${adminPaths.organizations}/${organization?.name || organization?.id}`}
+            href={detailsPath}
             onClick={(e) => {
               e.preventDefault();
-              onNavigate(
-                `/${adminPaths.organizations}/${organization?.name || organization?.id}`,
-              );
+              onNavigate(detailsPath);
             }}
             leadingIcon={